Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pullquote Block: Add font size and line height to the Pullquote block #29361

Closed
wants to merge 1 commit into from
Closed

Pullquote Block: Add font size and line height to the Pullquote block #29361

wants to merge 1 commit into from

Conversation

danieldudzic
Copy link

Description

Add support for font size and line height to the Pullquote Block

With this change, at the moment, the setting targets the font size of the citation, this should probably be changed to target the text.

How has this been tested?

Tested in TT1 Blocks

Screenshots

Pullquote

Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • I've tested my changes with keyboard and screen readers.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.

@github-actions
Copy link

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @danieldudzic! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other.

If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information.

@github-actions github-actions bot added the First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository label Feb 26, 2021
@scruffian
Copy link
Contributor

This works as expected, but it does add the controls to the block. I'm not sure what the core team feels about this...

@jasmussen jasmussen requested a review from oandregal March 1, 2021 12:25
@jasmussen
Copy link
Contributor

jasmussen commented Mar 1, 2021

This is what I see:

Screenshot 2021-03-01 at 13 27 52

I don't mind that personally, as the controls are added to the inspector. And presumably they would benefit from any typography panel related improvements we make as part of #27331, so no objections here.

Edit: Also: thank you for your first PR to the repository! Woop!

@youknowriad youknowriad added [Block] Pullquote Affects the Pullquote Block [Type] Feature New feature to highlight in changelogs. labels Mar 1, 2021
Base automatically changed from master to trunk March 1, 2021 15:46
@oandregal
Copy link
Member

👋 Code-wise, this is how we add support for style properties automatically, etc. It'd be good from this point of view.

However, when testing this, things don't work as expected. There are a variety of style clashes coming from Gutenberg and the theme. See:

210302-1716-test-pullquote-block-supports.mp4

There's also the question that this block has two text items: the content and the citation, but the block only has a single UI component for font size. It sounds to me that users should be able to tweak these things separately?

(Haven't tested line-height in depth).

@carolinan
Copy link
Contributor

Font size and line height has been added to the block. I believe it was done in #34064.

@carolinan carolinan closed this Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Pullquote Affects the Pullquote Block First-time Contributor Pull request opened by a first-time contributor to Gutenberg repository [Type] Feature New feature to highlight in changelogs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants